Skip to content

Comments

chore: merge from develop to preview#4437

Merged
jorbuedo merged 2 commits intopreviewfrom
develop
Dec 16, 2025
Merged

chore: merge from develop to preview#4437
jorbuedo merged 2 commits intopreviewfrom
develop

Conversation

@jorbuedo
Copy link
Contributor

@jorbuedo jorbuedo commented Dec 16, 2025

Note

Only include wallet-controlled addresses and the actual staking key when determining required signers and building Ledger Plutus payloads, skipping foreign keys.

  • Security/Signer validation:
    • common/signatureUtils.ts:
      • getAddressAddressing: returns null unless wallet controls the address; respects strict (partial=false) mode.
    • tx/ledger/plutus.ts:
      • Filters requiredSigners to include only paths for addresses controlled by the wallet.
    • tx/ledger/signers.ts:
      • Adds staking key path only if the required signer matches the wallet’s staking key hash.
      • For payment key signers, includes addressing only if the constructed base address is wallet-controlled; skips otherwise (throws in strict mode).

Written by Cursor Bugbot for commit ae4abe8. This will update automatically on new commits. Configure here.

@jorbuedo jorbuedo merged commit 64e45fc into preview Dec 16, 2025
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant